- Apr 29, 2023
-
-
Jacques Pienaar authored
Add method to set a desired bytecode file format to generate. Change write method to be able to return status including the minimum bytecode version needed by reader. This enables generating an older version of the bytecode (not dialect ops, attributes or types). But this does not guarantee that an older version can always be generated, e.g., if a dialect uses a new encoding only available at later bytecode version. This clamps setting to at most current version. Differential Revision: https://reviews.llvm.org/D146555
-
Tom Praschan authored
Differential Revision: https://reviews.llvm.org/D147847
-
NAKAMURA Takumi authored
To prevent race in `NATIVE`, let each action depend on preceding action. At the moment, this is restricted only for "Visual Studio". For example, - `llvm-tblgen-host` depends on the target's `llvm-tblgen`. - `clang-tblgen-host` depends on the target's `clang-tblgen` and `llvm-tblgen-host`. This is rework for D54153. `build_native_tool` has been introduced since D60024. Differential Revision: https://reviews.llvm.org/D149072
-
Jon Phillips authored
The option allows users to specify how many columns to use to indent the contents of initializer lists. Closes #51070. Differential Revision: https://reviews.llvm.org/D146101
-
Luo, Yuanke authored
The previous patch (D148980) didn't set the InstrIdxForVirtReg correctly in genAlternativeDpCodeSequence(). It causes vnni lit test failure when LLVM_ENABLE_EXPENSIVE_CHECKS is on.
-
Hans Wennborg authored
This broke the lit tests on Mac, see comment on the code review. > This change the types to match the ones used in: > Darwin/debug_external.cpp > debugging.cpp > > Reviewed By: vitalybuka > > Differential Revision: https://reviews.llvm.org/D148214 This reverts commit ea7d6e65.
-
Yeting Kuo authored
The patch disables strict node mutation by setting IsStrictFPEnabled to true. It also fixes wrong lowering action of strict_fsetcc(s). Reviewed By: craig.topper Differential Revision: https://reviews.llvm.org/D149320
-
NAKAMURA Takumi authored
-
Joseph Huber authored
The implementation of the test printing currently expects a null terminated C-string. However, the `write_to_stderr` interface uses a string view, which doesn't need to be null terminated. This patch changes the printing interface to directly use `fwrite` instead rather than relying on a null terminator. Reviewed By: sivachandra Differential Revision: https://reviews.llvm.org/D149493
-
Manna, Soumi authored
Reported by Coverity: AUTO_CAUSES_COPY Unnecessary object copies can affect performance 1. Inside "ODRHash.cpp" file, in clang::ODRHash::AddCXXRecordDecl(clang::CXXRecordDecl const *): Using the auto keyword without an & causes the copy of an object of type CXXBaseSpecifier. 2. Inside "Tokens.cpp" file, in clang::syntax::TokenBuffer::dumpForTests[abi:cxx11](): Using the auto keyword without an & causes the copy of an object of type DenseMapPair. 3. Inside "TargetID.cpp" file, in clang::getCanonicalTargetID[abi:cxx11](llvm::StringRef, llvm::StringMap<bool, llvm::MallocAllocator> const &): Using the auto keyword without an & causes the copy of an object of type pair. Reviewed By: tahonermann, aaron.ballman Differential Revision: https://reviews.llvm.org/D148639
-
Uday Bondhugula authored
Add canonicalization for affine.for bounds to use operand info (when operands are outer loop affine.for IVs) to simplify bounds: redundant bound expressions are eliminated in specific cases that are easy to check and well-suited for op canonicaliation. If the lowest or the highest value the affine expression can take is already covered by other constant bounds, the expression can be removed. Eg: `min (d0) -> (32 * d0 + 32, 32)(%i) where 0 <= %i < 2` The first expression can't be less than 32 and can be simplified away with a lightweight local rewrite. This simplification being part of canonicalization only handles simple expressions, specifically, a sum of products of operands with constants. This is a very common and a dominant case where such simplification is desired. These can be flattened without any local variables. Reviewed By: dcaballe, springerm Differential Revision: https://reviews.llvm.org/D149007
-
Vitaly Buka authored
Reviewed By: thurston Differential Revision: https://reviews.llvm.org/D149430
-
Vitaly Buka authored
Acidentally relanded without the fix. This reverts commit 008c3934.
-
Vitaly Buka authored
Reviewed By: thurston Differential Revision: https://reviews.llvm.org/D149430
-
Jason Molenda authored
Refactor the debugserver watchpiont support in anticipating of adding support for AArch64 MASK hardware watchpoints to watch larger regions of memory. debugserver already had support for handling a request to watch an unaligned region of memory up to 8 bytes using Byte Address Select watchpoints - it would split an unaligned watch request into two aligned doublewords that could be watched with two hardware watchpoints using the BAS specification. This patch generalizes that code for properly aligning, and possibly splitting, a watchpoint request into two hardware watchpoints to handle any size request. And separates out the specifics about BAS watchpoints into its own method, so a sibling method for MASK watchpoints can be dropped in next. Differential Revision: https://reviews.llvm.org/D149040 rdar://108233371
-
Jason Molenda authored
i386 and armv7 macOS/iOS cannot be built with current Xcode any longer; we cannot build or test the support code for running debugserver on these targets. Remove the code. Differential Revision: https://reviews.llvm.org/D149503
-
Jonas Devlieghere authored
This patch refactors the macOS implementation of OpenFileInExternalEditor. It fixes an AppleEvent memory leak, the caching of LLDB_EXTERNAL_EDITOR and speculatively fixes a crash when CFURL is NULL (rdar://108633464). The new code also improves error handling, readability and documents calls to the CoreFoundation Launch Services APIs. A bunch of the Launch Services APIs have been deprecated (LSFindApplicationForInfo, LSOpenURLsWithRole). The preferred API is LSOpenCFURLRef but it doesn't specifying the "location" Apple Event which is used to highlight the current line and switching over would regress the existing behavior. rdar://108633464 Differential revision: https://reviews.llvm.org/D149482
-
Vitaly Buka authored
Break internal tests, but it supposed to be almost NFC. This reverts commit bf12b746.
-
Craig Topper authored
-
Mahesh Ravishankar authored
Reviewed By: hanchung Differential Revision: https://reviews.llvm.org/D149120
-
LLVM GN Syncbot authored
-
Jie Fu authored
/Users/jiefu/llvm-project/llvm/lib/Object/GOFFObjectFile.cpp:199:3: error: use of undeclared identifier 'EC' EC = ConverterEBCDIC::convertToUTF8(SymbolName, SymbolNameConverted); ^ 1 error generated.
-
Vitaly Buka authored
c55fffec replaced fcmp with fpclass. ``` declare i1 @llvm.is.fpclass(<fptype> <op>, i32 <test>) declare <N x i1> @llvm.is.fpclass(<vector-fptype> <op>, i32 <test>) ``` Perfect fix will require checking bits of <op> corresponding to <test> argument. For now just propagate shadow without reporting before intrinsic. Still existing handling of fcmp is also simple OR, so it's not making it worse. Reviewed By: eugenis Differential Revision: https://reviews.llvm.org/D149491
-
Vitaly Buka authored
Msan uses strict approach for unknown intrinsics like fpclass. So any uninitialized bits in the value trigger msan report.
-
Yusra Syeda authored
This patch details the GOFF file format and implements the GOFFObjectfile class with support for only the HDR, ESD and END GOFF records. Reviewed By: jhenderson, kpn Differential Revision: https://reviews.llvm.org/D98437
-
Neumann Hon authored
This reverts commit 5b242318.
-
Neumann Hon authored
This patch details the GOFF file format and implements the GOFFObjectfile class with support for only the HDR, ESD and END GOFF records. Reviewed By: jhenderson, kpn Differential Revision: https://reviews.llvm.org/D98437
-
wlei authored
-
Slava Zakharin authored
When the default case requires block arguments, they have to be passed through the cf.br - this piece was missing. Reviewed By: clementval Differential Revision: https://reviews.llvm.org/D149484
-
Michael Jones authored
The time functions have not yet been updated to match our new coding patterns. This patch removes some unnecessary includes, adjusts the names of the test targets, and adds several TODO comments. It is my intention to follow this patch up with additional cleanup. Reviewed By: sivachandra, rtenneti Differential Revision: https://reviews.llvm.org/D149487
-
Craig Topper authored
This allows the user to set the size of the scalable vector so they can be used in structs and as the type of global variables. This works by representing the type as a fixed vector instead of a scalable vector in IR. Conversions to and from scalable vectors are made where necessary like function arguments/returns and intrinsics. This features has been requested here https://github.com/riscv-non-isa/rvv-intrinsic-doc/issues/176 I know arm_sve_vector_bits is used by the Eigen library so this could be used to port Eigen to RVV. This patch adds a new preprocessor define `__riscv_v_fixed_vlen` that is set when -mrvv_vector_bits is passed on the command line. The code is largely based on the AArch64 code. A lot of code was copy/pasted and then modiied to RVV. There may be some opportunities for sharing. This first patch only supports the LMUL=1 types. Additional changes will be needed to support other LMULs. I have also not supported mask vectors. Differential Revision: https://reviews.llvm.org/D145088
-
AdityaK authored
Reviewers: craig.topper, aeubanks, Peter Differential Revision: https://reviews.llvm.org/D134490
-
Craig Topper authored
-
Arthur Eubanks authored
-
Florian Hahn authored
Clarify variable name as suggested in D147964 to reduce diff.
-
Arthur Eubanks authored
-
Michael Jones authored
The differential fuzzer for atof found a bug in glibc's handling of hexadecimal rounding. Since we can't easily update glibc and we want to avoid false positives when running the fuzzer, I've added an exception to skip all hexadecimal subnormal cases. Reviewed By: sivachandra Differential Revision: https://reviews.llvm.org/D149359
-
Vitaly Buka authored
Reviewed By: thurston Differential Revision: https://reviews.llvm.org/D149430
-
Teresa Johnson authored
Switch to the just updated versions of the API in tcmalloc that change the name of the hot cold paramter to a reserved identifier __hot_cold_t. This was based on feedback from Richard Smith, as I also need to add some follow-on handling to clang so they are annotated properly. Differential Revision: https://reviews.llvm.org/D149475
-
Jakub Kuderski authored
This enabled more optimization opportunities by moving zero/sign-extension closer to the use. Reviewed By: antiagainst Differential Revision: https://reviews.llvm.org/D149282
-