Skip to content

Commit

Permalink
[GLUTEN-8375][CH][MINOR] Fix USE_EMBEDDED_COMPILER (#8441)
Browse files Browse the repository at this point in the history
  • Loading branch information
baibaichen authored Jan 7, 2025
1 parent eae63ae commit 198b1cc
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
#pragma once

#include "SparkFunctionDecimalBinaryOperator.h"

#include <Columns/ColumnDecimal.h>
Expand All @@ -27,7 +29,6 @@
#include <Functions/IFunction.h>
#include <Functions/castTypeToEither.h>
#include <Common/CurrentThread.h>
#include <Common/Stopwatch.h>

#if USE_EMBEDDED_COMPILER
#include <DataTypes/Native.h>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,9 +15,15 @@
* limitations under the License.
*/
#pragma once
#include "config.h"

#include <base/arithmeticOverflow.h>

#if USE_EMBEDDED_COMPILER
#include <DataTypes/Native.h>
#include <llvm/IR/IRBuilder.h>
#endif

#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wbit-int-extension"

Expand All @@ -27,11 +33,11 @@ using BitUInt128 = unsigned _BitInt(128);
using BitInt256 = signed _BitInt(256);
using BitUInt256 = unsigned _BitInt(256);
#else
// up to version 18, clang supports large _Bitint sizes on x86 and x86-64;
// up to version 18, clang supports large _Bitint sizes on x86 and x86-64;
// but on arm and aarch64, they are currently only supported up to 128 bits.
// https://stackoverflow.com/questions/78614816/why-am-i-getting-a-256-bit-arithmetic-error-unsigined-bitint-of-bit-sizes-gre
using BitInt256 = Int256;
using BitUInt256 = UInt256;
using BitInt256 = Int256;
using BitUInt256 = UInt256;
#endif

namespace local_engine
Expand Down

0 comments on commit 198b1cc

Please sign in to comment.