diff options
| author | Eric Fiselier <eric@efcs.ca> | 2018-02-02 19:58:34 +0000 |
|---|---|---|
| committer | Eric Fiselier <eric@efcs.ca> | 2018-02-02 19:58:34 +0000 |
| commit | 88df555d051e933b619e3679dcfde3756e61cc1f (patch) | |
| tree | 15a70d02eafbbe4806808445beeb2d83adc73e4a /clang/lib/CodeGen/CodeGenFunction.h | |
| parent | 66ce45150f2a4df0d7012b9c630862ad1db4d0cb (diff) | |
| download | bcm5719-llvm-88df555d051e933b619e3679dcfde3756e61cc1f.tar.gz bcm5719-llvm-88df555d051e933b619e3679dcfde3756e61cc1f.zip | |
Emit label names according to -discard-value-names.
Summary:
Previously, Clang only emitted label names in assert builds.
However there is a CC1 option -discard-value-names that should have been used to control emission instead.
This patch removes the NDEBUG preprocessor block and instead allows LLVM to handle removing the names in accordance with the option.
Reviewers: erichkeane, aaron.ballman, majnemer
Reviewed By: aaron.ballman
Subscribers: cfe-commits
Differential Revision: https://reviews.llvm.org/D42829
llvm-svn: 324127
Diffstat (limited to 'clang/lib/CodeGen/CodeGenFunction.h')
| -rw-r--r-- | clang/lib/CodeGen/CodeGenFunction.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/clang/lib/CodeGen/CodeGenFunction.h b/clang/lib/CodeGen/CodeGenFunction.h index 75071a8e322..844ccf00947 100644 --- a/clang/lib/CodeGen/CodeGenFunction.h +++ b/clang/lib/CodeGen/CodeGenFunction.h @@ -1849,11 +1849,7 @@ public: llvm::BasicBlock *createBasicBlock(const Twine &name = "", llvm::Function *parent = nullptr, llvm::BasicBlock *before = nullptr) { -#ifdef NDEBUG - return llvm::BasicBlock::Create(getLLVMContext(), "", parent, before); -#else return llvm::BasicBlock::Create(getLLVMContext(), name, parent, before); -#endif } /// getBasicBlockForLabel - Return the LLVM basicblock that the specified |
