[Lldb-commits] [PATCH] D77107: [intel-pt] Implement a basic test case

walter erquinigo via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Apr 1 10:26:35 PDT 2020

wallace marked an inline comment as done.
wallace added inline comments.

Comment at: lldb/test/API/tools/intel-features/intel-pt/test/main.cpp:2-8
+//// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+//// See https://llvm.org/LICENSE.txt for license information.
+//// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
labath wrote:
> wallace wrote:
> > labath wrote:
> > > We're not putting license headers on tests.
> > > 
> > > (Do these get automatically created by some IDEs or something? Can they be configured not to do that?)
> > I just copy pasted it from another test
> Which one? I don't see any test like that in the lldb repo. Was it from the swift fork or something?
I copied it from here https://github.com/llvm/llvm-project/blob/master/lldb/tools/intel-features/intel-mpx/test/main.cpp
I'll remove the header from that file as well

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list