[all-commits] [llvm/llvm-project] 291b41: [Misc] Use `LLVM_ENABLE_ABI_BREAKING_CHECKS` corre...

[paperchalice via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 291b415c6c39156c82c7cdefd7a6a67657fb6927
      https://github.com/llvm/llvm-project/commit/291b415c6c39156c82c7cdefd7a6a67657fb6927
  Author: paperchalice <liujunchang97 at outlook.com>
  Date:   2024-06-10 (Mon, 10 Jun 2024)

  Changed paths:
    M llvm/include/llvm/Passes/StandardInstrumentations.h
    M llvm/include/llvm/Support/GenericDomTreeConstruction.h
    M llvm/include/llvm/Transforms/Scalar/JumpThreading.h
    M llvm/include/llvm/Transforms/Scalar/LoopPassManager.h
    M llvm/include/llvm/Transforms/Utils/ScalarEvolutionExpander.h
    M llvm/lib/Passes/StandardInstrumentations.cpp
    M llvm/lib/Transforms/Utils/ScalarEvolutionExpander.cpp

  Log Message:
  -----------
  [Misc] Use `LLVM_ENABLE_ABI_BREAKING_CHECKS` correctly (#94212)

`LLVM_ENABLE_ABI_BREAKING_CHECKS` is always defined:

https://github.com/llvm/llvm-project/blob/72c901f5e59477e568b1b04dae9de753b9d1f6f3/llvm/include/llvm/Config/abi-breaking.h.cmake#L16C2-L16C15
It uses `cmakedefine01` rather than `cmakedefine`, so
`LLVM_ENABLE_ABI_BREAKING_CHECKS` is always defined,
so the preprocessed code is probably not what the author wanted.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications